STM32WBAX PM rework to support STOP/STDBY modes #92847
Merged
+162
−107
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
asm5878
changed the title
|
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
github-actions
bot
added
manifest
manifest-hal_stm32
DNM (manifest)
msmttchr
reviewed
Jul 8, 2025
There was a problem hiding this comment.
Hello @asm5878, thanks a lot for this draft PR, very interesting proposal. I added few preliminary comments, and I would like also to involve @HoZHel and @FrancescoCiminoST
msmttchr
reviewed
Jul 20, 2025
asm5878
force-pushed
the
STM32WBAX_PM_Enhancements
branch
from
e119b24 to
a58e6c7
Compare
MBrandt-Z
reviewed
Aug 7, 2025
MBrandt-Z
reviewed
Aug 7, 2025
asm5878
force-pushed
the
STM32WBAX_PM_Enhancements
branch
3 times, most recently
from
ac8ec9f to
b1e2b46
Compare
zephyrbot
added
area: Bluetooth Host
asm5878
force-pushed
the
STM32WBAX_PM_Enhancements
branch
from
c8a0cd8 to
67e08da
Compare
asm5878
requested review from
erwango,
etienne-lms,
mathieuchopstm and
msmttchr
msmttchr
approved these changes
Sep 2, 2025
asm5878
force-pushed
the
STM32WBAX_PM_Enhancements
branch
from
67e08da to
11e0232
Compare
mathieuchopstm
previously approved these changes
Sep 3, 2025
etienne-lms
previously approved these changes
Sep 3, 2025
erwango
requested changes
Sep 4, 2025
west.yml
Outdated
| - hal | ||
| - name: hal_stm32 | ||
| revision: 611d54c237623ea71a367502bfbd736fc67ba15f | ||
| revision: pull/294/head |
There was a problem hiding this comment.
3a4b521441607646ec43bece2c53f333ce7f9b69
There was a problem hiding this comment.
Thanks a lot @erwango . Done
There was a problem hiding this comment.
Conflict.... Rebase on going
There was a problem hiding this comment.
Rebased and conflict solved
asm5878
dismissed stale reviews from etienne-lms and mathieuchopstm
via
0201e99
asm5878
force-pushed
the
STM32WBAX_PM_Enhancements
branch
from
11e0232 to
0201e99
Compare
github-actions
bot
removed
the
DNM (manifest)
soc power management rework to support power states removing dependency on ST system clock manager (SCM). Signed-off-by: Alessandro Manganaro <[email protected]>
Updating standby exit latency time according worst case value from stm32wba5 and stm32wba6 datasheets. Signed-off-by: Alessandro Manganaro <[email protected]>
asm5878
force-pushed
the
STM32WBAX_PM_Enhancements
branch
from
0201e99 to
b4e6338
Compare
|
Please retry analysis of this Pull-Request directly on SonarQube Cloud |
erwango
approved these changes
Sep 4, 2025
etienne-lms
approved these changes
Sep 4, 2025
This was referenced Sep 5, 2025
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Aim of this PR is a cleaner support of stop/stdby modes for stm32wbax socs.
System power management has been reworked to remove:
Scope of this PR has been reduced to manage only the pm at system level.
A new PR (#95151) has been created to update the pm for the stm32wbax ble hci driver.
FYI: @Chastillon